Skip to content

ci(ocr review): bump to v1.12.11 and add update lock - #48

Merged
BasuruK merged 1 commit into
masterfrom
feature/basuruk/pipeline-update-for-code-review
Oct 1, 2026
Merged

BasuruK merged 1 commit into
masterfrom
feature/basuruk/pipeline-update-for-code-review

Conversation

@BasuruK

@BasuruK BasuruK commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

User description

bump the alibaba/open-code-review action from v1.12.9 to v1.12.11, and add the OCR_NO_UPDATE=1 environment variable to prevent the ocr launcher's self-updater from unlinking /usr/local/bin/ocr mid-run, which causes concurrent ocr config set commands to fail with MODULE_NOT_FOUND errors


PR Type

Bug fix, Configuration changes


Description

  • Add OCR_NO_UPDATE=1 env var to block self-updater

  • Bump alibaba/open-code-review action to v1.12.11

  • Pin ocr_version input to 1.12.11


Diagram Walkthrough

flowchart LR
  A["ocr_review.yml job"] --> B["env: OCR_NO_UPDATE=1"]
  A --> C["uses: alibaba/open-code-review@v1.12.11"]
  C --> D["ocr_version: 1.12.11"]
  B --> E["Prevent mid-run unlink of /usr/local/bin/ocr"]
Loading

File Walkthrough

Relevant files
Bug fix
ocr_review.yml
Pin ocr action to v1.12.11 and disable updater                     

.github/workflows/ocr_review.yml

  • Added OCR_NO_UPDATE: "1" job-level env with a comment explaining it
    blocks the OCR launcher's detached self-updater from racing with
    concurrent ocr config set calls
  • Bumped alibaba/open-code-review action SHA and ocr_version input from
    1.12.9 to 1.12.11
+9/-2     

bump the alibaba/open-code-review action from v1.12.9 to v1.12.11, and add the OCR_NO_UPDATE=1 environment variable to prevent the ocr launcher's self-updater from unlinking /usr/local/bin/ocr mid-run, which causes concurrent `ocr config set` commands to fail with MODULE_NOT_FOUND errors
@BasuruK BasuruK self-assigned this Oct 1, 2026
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 8687828a-9b21-4240-8484-463f400b2864

📥 Commits

Reviewing files that changed from the base of the PR and between 9379186 and f046892.

📒 Files selected for processing (1)
  • .github/workflows/ocr_review.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (2)
  • GitHub Check: Codacy Static Code Analysis
  • GitHub Check: code-review
🔇 Additional comments (1)
.github/workflows/ocr_review.yml (1)

16-22: LGTM!

Also applies to: 117-120


Summary by CodeRabbit

  • Chores
    • Updated the automated code review tooling and its configuration. These changes affect the review process and do not add or change user-facing features.

Walkthrough

The code-review job now sets OCR_NO_UPDATE to "1" for its steps. The workflow also updates the OpenCodeReview action and ocr_version pin from v1.12.9 to v1.12.11.

Changes

OCR review workflow

Layer / File(s) Summary
Code-review workflow settings
.github/workflows/ocr_review.yml
The job sets OCR_NO_UPDATE to "1" for its steps. The workflow updates the OpenCodeReview action and ocr_version pin to v1.12.11.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to f0468

The review workflow disables the updater that could interfere with OCR configuration, so no merge-blocking issue is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to f0468

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/workflows/ocr_review.yml: The job now sets OCR_NO_UPDATE to "1" for its steps.
  • observed — Modified behavior in .github/workflows/ocr_review.yml: The workflow now uses OpenCodeReview action v1.12.11 and sets ocr_version to "1.12.11" instead of v1.12.9 and "1.12.9".
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description accurately explains the action version update and the addition of OCR_NO_UPDATE=1 to prevent self-updater conflicts.
Title check ✅ Passed The title clearly summarizes both main changes: the OpenCodeReview action upgrade and the update lock environment setting.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
✨ Simplify code
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

A rabbit checks the workflow file,
And sees the version hop in style.
“No update,” says the setting bright,
The review steps now run just right.
Then off it bounds beneath the moon,
With carrots for a job well-tuned.

Comment @coderabbitai help to get the list of available commands.

@joe-abey

joe-abey Bot commented Oct 1, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@joe-abey

joe-abey Bot commented Oct 1, 2026

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@opencodereview-ocr-bot

opencodereview-ocr-bot Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

🔍 OpenCodeReview found 1 issue(s) in this PR.

  • ✅ Successfully posted inline: 1 comment(s)

Comment thread .github/workflows/ocr_review.yml
@BasuruK
BasuruK merged commit f50f84d into master Oct 1, 2026
3 checks passed
@BasuruK
BasuruK deleted the feature/basuruk/pipeline-update-for-code-review branch October 1, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant